Skip to content

Conversation

@kit-ty-kate
Copy link
Member

Required for #6611

Labeling every single OpamConsole.confirm is a bit too large of a task at the moment but this can be done in the future incrementally as need arise (e.g. #6611 does add three labels).

For now i prefer advertising this feature as internal-use only to allow it to grow and evolve as needed, rather than inviting users to use it and expect a full feature support with menus and other interactive prompts. In time though i'm sure it'll become something usable more widely.

Copy link
Collaborator

@rjbou rjbou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed in meeting, this PR is good to go to unlock #6611.
In #6611, we'll introduce what is strictly needed for the tests. In a following PR, we'll implement completely a first real support for this option.

@kit-ty-kate kit-ty-kate merged commit ce734b3 into ocaml:master Oct 6, 2025
57 checks passed
@kit-ty-kate kit-ty-kate deleted the auto-answer branch October 6, 2025 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants